Skip to content

Conversation

MintsInc
Copy link
Member

@MintsInc MintsInc commented Sep 25, 2025

Context

The concurrency group of the integration workflow is currently the same for all client CI when ran from a PR on the specification repo. This causes all integration test jobs but one to be cancelled.

To prevent this we change the concurrency group so it's dependent on the branch name and the targeted client.

There was also duplication of conditions to run the workflow which is unnecessary and might cause issues down the line.

…ix integration concurrency group

- Remove duplicate conditions from reusable workflows that don't work properly in
  workflow_call context since github.event.pull_request is not available
- Fix integration test concurrency group to be client-specific
- The main test.yml workflow already handles these conditions correctly
@MintsInc MintsInc merged commit da3e469 into master Sep 25, 2025
12 checks passed
@MintsInc MintsInc deleted the ulysse.mavrocordatos/fix-integration-concurrency-group branch September 25, 2025 09:27
github-actions bot pushed a commit that referenced this pull request Sep 25, 2025
…ix integration concurrency group (#2839)

- Remove duplicate conditions from reusable workflows that don't work properly in
  workflow_call context since github.event.pull_request is not available
- Fix integration test concurrency group to be client-specific
- The main test.yml workflow already handles these conditions correctly da3e469
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants